Feature: Limit Preis als Attribut #1330
Closed
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
buchen
requested changes
Jan 5, 2020
There was a problem hiding this comment.
Hi @debegr92,
vielen Dank für den Change. Das ist schon ein ziemlich alter Feature Wunsch. Und die Idee mit dem Comparator (">=") direkt im Eingabefeld finde ich super. Der Code ist super. Ich habe noch ein paar Anmerkungen die vor allem aus der PP Code Historie kommen.
Wichtig wäre mir
- der letzte Kurs kann null sein
- das Pattern sollte alle Sprachvarianten unterstützen
- das Attribut darf nur für Wertpapiere angelegt werden können
- Versionsnummer in Client/ClientFactory und XML Name
Andreas.
name.abuchen.portfolio.ui/src/name/abuchen/portfolio/ui/views/columns/AttributeColumn.java
Outdated
Show resolved
Hide resolved
name.abuchen.portfolio.ui/src/name/abuchen/portfolio/ui/views/columns/AttributeColumn.java
Outdated
Show resolved
Hide resolved
name.abuchen.portfolio/src/name/abuchen/portfolio/model/AttributeType.java
Outdated
Show resolved
Hide resolved
name.abuchen.portfolio.ui/src/name/abuchen/portfolio/ui/views/settings/AttributeFieldType.java
Show resolved
Hide resolved
name.abuchen.portfolio/src/name/abuchen/portfolio/money/LimitPrice.java
Outdated
Show resolved
Hide resolved
buchen
added a commit
that referenced
this pull request
Jan 6, 2020
- Fixed parsing of relational operator - Fixed comparison when coloring cell background - Renamed 'CompareType' to 'RelationalOperator' - Right-aligned limit column - Limit creation of limit to attributes attached to securities Issue: #1330
|
Hi @debegr92, ist drin! Vielen Dank. Ich habe Deine commits ge"squashed" und auf den master ge"rebased". Ein paar Punkte habe ich noch angepasst - siehe 26e915a Wenn Du Deine persönlich Datei schon angepasst hast, dann müsstest Du "compareType" noch in "operator" im XML umbenennen. Deine Lösung gefällt mir richtig gut. Vor allem die Idee das Vergleichszeichen direkt in der Eingabe mit aufzunehmen. Andreas. |
cmaoling
pushed a commit
to cmaoling/portfolio
that referenced
this pull request
Feb 18, 2020
Issue: portfolio-performance#1330 Issue: portfolio-performance#143 Signed-off-by: Dennis Greguhn <dennis@greguhn.de> [squashed commits; reformatted to PP code style; rebased to master] Signed-off-by: Andreas Buchen <andreas.buchen@gmail.com>
cmaoling
pushed a commit
to cmaoling/portfolio
that referenced
this pull request
Feb 18, 2020
- Fixed parsing of relational operator - Fixed comparison when coloring cell background - Renamed 'CompareType' to 'RelationalOperator' - Right-aligned limit column - Limit creation of limit to attributes attached to securities Issue: portfolio-performance#1330
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Nach der etwas angestaubten Unterhaltung hier #143, möchte ich meine Lösung präsentieren:
Ein neuer AttributeType "LimitPrice" wurde eingeführt. Dieser akzeptiert die Vergleichs-Operatoren >=, <=, > und < gefolgt von einem Preis.
Unterschreitungen werden rot, Überschreitungen grün dargestellt.